Skip to content
This repository was archived by the owner on Apr 5, 2022. It is now read-only.

DataTable, TopNav, and Modal CSS changes#333

Merged
Kadinvanvalin merged 28 commits intomasterfrom
CJPM-17455
Apr 12, 2021
Merged

DataTable, TopNav, and Modal CSS changes#333
Kadinvanvalin merged 28 commits intomasterfrom
CJPM-17455

Conversation

@lgrinta-cj
Copy link
Copy Markdown

@lgrinta-cj lgrinta-cj commented Apr 6, 2021

  • Update to CONTRIBUTING.md about installing lerna.
  • BlankSlate: Remove hard-coded / default icon graphic and white background Panel per UX.
  • Form: Remove 24px margin between fields per UX.
  • Modal: Remove header and footer boarder lines, push top down, add left & right padding of 25px, include a -md class which is not quite as wide as the default. All per UX.
  • DataTable: Change the thickness and color of the header row bottom border, remove the extra DataTable.css import which was causing issues, added alignment argument to header props. All per UX.
  • TopNav: Fixed some sizing and cursor issues.

@Kadinvanvalin
Copy link
Copy Markdown
Contributor

Do you mind running eslint on the files you changed? You didn't add any new issues, but TopNav and DataTable could use some cleanup.

@joshhidley
Copy link
Copy Markdown
Contributor

@Kadinvanvalin Done. Thanks!

@Kadinvanvalin
Copy link
Copy Markdown
Contributor

Looks good to me!

@Kadinvanvalin Kadinvanvalin merged commit 70a4cce into master Apr 12, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants